Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

Few more possible regular expressions to check checkbox is checked #57

Merged
merged 2 commits into from Mar 9, 2013
Merged

Few more possible regular expressions to check checkbox is checked #57

merged 2 commits into from Mar 9, 2013

Conversation

ghost
Copy link

@ghost ghost commented Dec 11, 2012

I added a couple of regular expressions that helps the user to check if a checkbox is checked or not on a page.

Committed here - de37a1a

Is it worth?

@@ -381,6 +383,9 @@ public function assertCheckboxChecked($checkbox)
* Checks, that checkbox with specified in|name|label|value is unchecked.
*
* @Then /^the "(?P<checkbox>(?:[^"]|\\")*)" checkbox should not be checked$/
* @Then /^I (?:|should )see the checkbox "(?P<checkbox>(?:[^"]|\\")*)" (?:is not checked|is unchecked)$/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not look like an English sentence. you have 2 verbs

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right. The we can remove "I should see .... " from the above regular expression and keep the rest.??

@stof
Copy link
Member

stof commented Dec 11, 2012

And the trait should be kept in sync with the class

@ghost
Copy link
Author

ghost commented Dec 13, 2012

Could you please point it out at one example here? I am not able to figure out the trait. :)

(Sundar is new here and is trying to learn new stuff on git as much as possible. Please excuse.)

@stof
Copy link
Member

stof commented Dec 13, 2012

you have to update this file to keep the same step definitions

@ghost
Copy link
Author

ghost commented Dec 13, 2012

I have updated as suggested and the commit is done here - b6df191

I above commit is on top of my previous commit de37a1a. Is this fine or should I do fork it again and commit with only the new changes?

everzet added a commit that referenced this pull request Mar 9, 2013
Few more possible regular expressions to check checkbox is checked
@everzet everzet merged commit 523f2e7 into Behat:master Mar 9, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants